Skip to content

Hotfix k8s bq updater#41

Merged
monoid-is-a-monad merged 8 commits into
releasedfrom
hotfix-k8s-bq-updater
Jun 9, 2026
Merged

Hotfix k8s bq updater#41
monoid-is-a-monad merged 8 commits into
releasedfrom
hotfix-k8s-bq-updater

Conversation

@monoid-is-a-monad

Copy link
Copy Markdown
Contributor

Mitigates:
#37

Notes:
Cronjobs are enabled so when we deploy this, cronjobs will start to run
There are 2 secrets intentially missing

@monoid-is-a-monad monoid-is-a-monad requested a review from CodiePP May 27, 2026 17:23

@CodiePP CodiePP left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a minor thing: we can use kustomization to concretize the image

Comment thread deployment/bq-updater/cronjob.yaml Outdated
Comment thread deployment/bq-updater/kustomization.yaml Outdated
Comment thread deployment/bq-updater/cronjob.yaml Outdated
app: cardano-bq-slot-sync
spec:
suspend: true
schedule: "3,33 * * * *"

@CodiePP CodiePP Jun 2, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should only run it once per hour at hh:34

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and, it would be easier if we use the timezone UTC:

  timeZone: Etc/UTC

Comment thread deployment/bq-updater/cronjob.yaml Outdated
app: cardano-bq-epoch-sync
spec:
suspend: true
schedule: "16 5 * * *"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this currently runs at 01:10 UTC.

and, it would be easier if we use the timezone UTC:

  timeZone: Etc/UTC

@monoid-is-a-monad monoid-is-a-monad merged commit a9558f9 into released Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants